Skip to content

Upgrade Sourcemeta dependencies#789

Merged
jviotti merged 2 commits into
mainfrom
new-deps-7
Jul 1, 2026
Merged

Upgrade Sourcemeta dependencies#789
jviotti merged 2 commits into
mainfrom
new-deps-7

Conversation

@jviotti

@jviotti jviotti commented Jul 1, 2026 •
edited by cubic-dev-ai Bot
Loading

Copy link
Copy Markdown
Member

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Review in cubic

@jviotti
Upgrade Sourcemeta dependencies
395154d 
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Jul 1, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 105 files

Re-trigger cubic

@augmentcode

augmentcode Bot commented Jul 1, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR updates the repository’s vendored Sourcemeta dependencies (Core/Blaze/JsonBinPack) to newer revisions and aligns local build integration with those upstream changes.

Changes:

  • Bump pinned revisions in top-level and vendored DEPENDENCIES files (Core, Blaze, JsonBinPack).
  • Adjust CMake integration to match new Core options, removing GoogleTest contrib wiring and introducing Core’s new test framework targets.
  • Blaze compiler: refine annotation enabling/collection semantics to control applicator short-circuiting.
  • Core HTTP: add http_cache_control_max_age() to parse Cache-Control max-age.
  • Core JSON-LD: add annotation-map driven jsonld_materialize() for expanded JSON-LD output.
  • Core Unicode/Numeric/Text: add UTF-32→UTF-8 encoders, out-of-range numeric errors, ULP-based float equality, and additional text helpers.

Notes: The large diff is primarily upstream vendored code churn (including removal of vendored GoogleTest sources).

🤖 Was this summary useful? React with 👍 or 👎

augmentcode[bot]
augmentcode Bot reviewed Jul 1, 2026
View reviewed changes

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread vendor/core/src/lang/test/test.cc

auto print_diagnostic(std::string_view message) -> void {
std::size_t start{0};
while (start <= message.size()) {

@augmentcode augmentcode Bot Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vendor/core/src/lang/test/test.cc:48: while (start <= message.size()) will emit an extra empty # diagnostic line if message ends with a trailing newline (since start reaches message.size()). Consider tightening the loop condition so the final iteration doesn’t print a zero-length line.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Comment thread vendor/core/src/core/http/cache_control_max_age.cc
return;
}

found = true;

@augmentcode augmentcode Bot Jul 1, 2026 •
edited
Loading

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vendor/core/src/core/http/cache_control_max_age.cc:32: found is set before validating the parsed max-age value, so a malformed first max-age directive prevents considering any subsequent valid max-age directives in the same header value. Consider only setting found after parsing succeeds (or documenting that a malformed first directive invalidates the whole header).

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Comment thread vendor/core/src/core/jsonld/jsonld_materialize.cc
@jviotti
Fix
c8d38ee 
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 5ee554a into main Jul 1, 2026
15 checks passed
@jviotti jviotti deleted the new-deps-7 branch July 1, 2026 18:22
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jul 9, 2026
build(deps): update sourcemeta/jsonschema to v16.1.0
af3c8bd 
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [sourcemeta/jsonschema](https://github.com/sourcemeta/jsonschema) | minor | `v16.0.0` → `v16.1.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>sourcemeta/jsonschema (sourcemeta/jsonschema)</summary>

### [`v16.1.0`](https://github.com/sourcemeta/jsonschema/releases/tag/v16.1.0)

[Compare Source](sourcemeta/jsonschema@v16.0.0...v16.1.0)

#### What's Changed

- Update the README banner by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;783](sourcemeta/jsonschema#783)
- Rename `SOURCEMETA_JSONSCHEMA_CURL_SO` to `SOURCEMETA_CORE_CURL_SO` by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;784](sourcemeta/jsonschema#784)
- Fix some validation edge cases due to over optimisations in `validate --fast` by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;785](sourcemeta/jsonschema#785)
- Fix YAML syntax errors not properly caught by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;787](sourcemeta/jsonschema#787)
- Various minor fixes and performance optimisations by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;790](sourcemeta/jsonschema#790), [#&#8203;789](sourcemeta/jsonschema#789), [#&#8203;788](sourcemeta/jsonschema#788), and [#&#8203;786](sourcemeta/jsonschema#786)
- Implement an `rdf` command for converting into JSON-LD by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;791](sourcemeta/jsonschema#791)

**Full Changelog**: <sourcemeta/jsonschema@v16.0.0...v16.1.0>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yNTYuMCIsInVwZGF0ZWRJblZlciI6IjQzLjI1Ni4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiLCJhdXRvbWF0aW9uOmJvdC1hdXRob3JlZCIsImRlcGVuZGVuY3ktdHlwZTo6bWlub3IiXX0=-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@augmentcode augmentcode[bot] augmentcode[bot] left review comments
@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@jviotti